Skip to content

Conversation

@antellezr-mdb
Copy link
Collaborator

@antellezr-mdb antellezr-mdb commented Sep 11, 2025

@antellezr-mdb antellezr-mdb requested a review from a team as a code owner September 11, 2025 00:04
@antellezr-mdb antellezr-mdb changed the title (DOCSP-53435) Update flex-cluster README DOCSP-53435: Update flex-cluster README Sep 11, 2025
@antellezr-mdb antellezr-mdb changed the title DOCSP-53435: Update flex-cluster README doc: DOCSP-53435: Update flex-cluster README Sep 11, 2025
@antellezr-mdb antellezr-mdb self-assigned this Sep 11, 2025
Copy link
Member

@lantoli lantoli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

a couple of comments, otherwise LGTM, thanks!

@lantoli
Copy link
Member

lantoli commented Sep 11, 2025

@antellezr-mdb you can see a PR check is failing: "Code Health / check-l1-updated".

It should be fixed if you bring latest changes from master to your PR branch, let me know if you need any help.

(at the end we need to have all PR checks pass before merging)

Copy link
Collaborator

@marcosuma marcosuma left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM after addressing Leo's comments

Copy link
Collaborator

@xargom xargom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@antellezr-mdb
Copy link
Collaborator Author

Comments addressed

Copy link
Member

@lantoli lantoli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, feel free to merge once the last comments are addressed

@antellezr-mdb antellezr-mdb merged commit 29f49e3 into main Sep 12, 2025
14 checks passed
@antellezr-mdb antellezr-mdb deleted the DOCSP-53435 branch September 12, 2025 17:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants